infra: 코드 포매팅 양식 코드 전체 적용 #1031
Conversation
Test Results Summary 54 files 54 suites 15s ⏱️ Results for commit 2bc7dae. |
Migration Result 📦
Schema Validation Result 🔍✅ No schema mismatches detected. Results for commit ♻️ This comment has been updated with latest results. |
Summary of ChangesHello @KEEKE132, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 코드베이스 전체에 걸쳐 포매팅을 적용하고, 코드 가독성을 향상시키기 위한 리팩토링을 수행합니다. 주요 변경 사항으로는 Spotless 설정 업데이트, 메서드 시그니처 정렬, Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
전체 코드에 포매팅을 적용하는 PR이네요. spotless와 Eclipse Formatter 설정을 통해 일관된 코드 스타일을 유지하려는 노력이 좋습니다. 변경 사항 대부분이 포매팅과 관련된 것으로 보이며, 전체적으로 잘 적용된 것 같습니다. 다만, CouncilFeeExcelService.java 파일에서 포매터가 List.of를 두 줄로 나누면서 가독성이 다소 저하된 부분이 있어 리뷰 코멘트를 남겼습니다. 해당 부분에 대한 포매터 설정을 확인해보시면 좋을 것 같습니다.
| private static final Function<UserCouncilFeeResponseDto, List<String>> cellMappingFunction = userCouncilFee -> List | ||
| .of( |
There was a problem hiding this comment.
포매터가 List.of(...)를 두 줄로 나누면서 가독성이 다소 저하된 것 같습니다. 람다 표현식과 함께 한 줄로 작성하거나, 이전처럼 userCouncilFee -> 부분을 다음 줄로 내리는 것이 더 자연스러워 보입니다. 포매터 설정을 확인하여 이 부분을 조정할 수 있는지 검토해 보시는 것을 추천합니다.
| private static final Function<UserCouncilFeeResponseDto, List<String>> cellMappingFunction = userCouncilFee -> List | |
| .of( | |
| private static final Function<UserCouncilFeeResponseDto, List<String>> cellMappingFunction = | |
| userCouncilFee -> List.of( |
🚩 관련사항
Close: #1030
📢 전달사항
다른 Pr을 적용하기 전 코드 전체에 포매팅을 적용했습니다.
또한 Eclipse Formatter 사용시 포매팅 일치를 위해 설정을 조금 수정하였습니다.
📸 스크린샷
spotlessCheck 성공 화면
📃 진행사항
⚙️ 기타사항
IDE 설정 가이드입니다.
(https://www.notion.so/Spotless-IDE-2db3d138d33c80bfab99e6abba8252a5?source=copy_link)
개발기간: